fix(seo): immediate EEAT and crawl-surface fixes by julianken · Pull Request #393 · julianken/detached-node

julianken · 2026-05-17T14:31:26Z

Summary

Six small independent fixes that stop active EEAT damage and remove crawl-surface bugs. Identity-neutral (no Tier C dependencies).

Changes

Fixed broken sameAs URL — src/lib/schema/config.ts:44: github.com/detached-node (HTTP 404) -> github.com/julianken (HTTP 200). A broken external link in schema is worse than no link — it signals an unverifiable entity to Google's entity graph and LLM citation pipelines.
Removed hardcoded detached-node.com in two OG image footers (opengraph-image.tsx line 109 and [slug]/opengraph-image.tsx line 131). Now derives from process.env.NEXT_PUBLIC_SERVER_URL with .dev fallback.
Lifted overviewLead out of <DisclosureSection> in pattern pages. The first paragraph of pattern body content was inside an expanded <details> element; <details> is in the DOM and Googlebot indexes it, but non-Google AI crawlers (GPTBot, ClaudeBot, PerplexityBot) have inconsistent rendering of collapsed content. Plain <p> eliminates the ambiguity.
Removed priority from sitemap (Google ignores since 2017 per John Mueller, Search Central). Keeps lastModified and changeFrequency.
Added CONTACT_EMAIL to src/lib/site-config.ts — used by Category E's RSS feed upgrade and future schema work.
Surfaced visible dateModified on pattern pages — already in JSON-LD, now rendered as <time> for human freshness signal.

Test plan

pnpm lint (including lint:adp)
pnpm test:unit
pnpm typecheck
CI E2E (will run on PR)
Visual check via pnpm dev of a representative pattern page (confirmed overviewLead renders as plain <p> outside details)

Coordination

The sameAs fix in this PR is a strict subset of issue feat(seo): migrate to Tier C identity (real name + bylines + ProfilePage schema) #388 (Tier C migration), which replaces the entire AUTHOR_CONFIG. If both merge, feat(seo): migrate to Tier C identity (real name + bylines + ProfilePage schema) #388 wins via three-way merge of the array (1-item -> 2-item). If feat(seo): migrate to Tier C identity (real name + bylines + ProfilePage schema) #388 merges first, this PR's sameAs change becomes a no-op on rebase.

Closes #387

- Fix broken github.com/detached-node sameAs URL -> github.com/julianken (HTTP 404 -> 200) - Replace hardcoded detached-node.com in OG image footers with runtime SITE_DOMAIN - Lift overviewLead out of DisclosureSection so AI crawlers index it unconditionally - Remove priority field from sitemap (Google has ignored it since 2017) - Add CONTACT_EMAIL to site-config (for RSS upgrade in Category E) - Surface visible dateModified on pattern pages (freshness signal) Closes #387

julianken-bot

Verdict: APPROVE

Six independent SEO fixes, all narrowly scoped, all consistent with their stated rationale. Diff is 23 additions across 7 files; well below the length threshold for an attention dropoff and easy to audit.

Verification ledger

Check	Command	Result
HEAD pinned	`gh pr view 393 --json headRefOid`	`52ab7396726310baaac04d0e4a59b6497ce101f2`
`pnpm typecheck`	`tsc --noEmit`	clean — exit 0, no errors
`pnpm test:unit`	`vitest run`	31 files / 525 tests passed
`pnpm lint:adp`	typecheck-sketches + validate-references + check-affiliate-links + lint-changelog	all OK
`pnpm lint` (eslint)	1435 errors / 66638 warnings — none in PR-touched files	pre-existing baseline (R7)
`github.com/julianken` HTTP status	`curl -sI`	200 (was 404 for `detached-node`) — the schema sameAs fix is genuine
`Next.js MetadataRoute.Sitemap.priority` optionality	context7 `/vercel/next.js/v16.1.6` docs	confirmed optional; removal is type-safe
Hardcoded `detached-node.com` left at HEAD	`git grep`	0 matches (the one in `blog-posting.ts` is `.dev` in a comment)

Findings

SUGGESTION — src/app/(frontend)/agentic-design-patterns/opengraph-image.tsx:25-27 / [slug]/opengraph-image.tsx:27-29: duplicate SITE_DOMAIN fallback constant across two files. See inline comment.

Bottom line

Sitemap priority removal: type-safe per Next 16 docs.
sameAs URL: I verified https://github.com/julianken returns 200 and https://github.com/detached-node returns 404. The fix is real.
overviewLead lift: removes the collapsed-by-default rendering ambiguity for non-Google AI crawlers; the <DisclosureSection> was already defaultOpen so visible content is unchanged for human readers; the label "Overview · 1-paragraph mechanism" is now absent which is a deliberate trade.
<time dateTime> element: pattern.dateModified is typed string (ISO) in types.ts:143, so the dateTime attribute receives a valid value.
CONTACT_EMAIL: currently unused; the PR body frames it as forward groundwork for upcoming PRs. Acceptable.

Same-tier note

I am running as opus. If the implementer ran on opus as well, this review has same-tier perplexity-familiarity exposure (arxiv 2410.21819). I did a mandatory R8 second pass with the "this contains at least one improvement" prior; the duplicate-constant SUGGESTION is what surfaced, and I am holding it at SUGGESTION rather than escalating.

— @julianken-bot

julianken-bot · 2026-05-17T14:54:54Z


+const SITE_DOMAIN = process.env.NEXT_PUBLIC_SERVER_URL
+  ? new URL(process.env.NEXT_PUBLIC_SERVER_URL).hostname
+  : "detached-node.dev";


Both [slug]/opengraph-image.tsx and opengraph-image.tsx now carry an identical SITE_DOMAIN block with the same "detached-node.dev" literal. If the canonical domain ever changes — or a future env-handling refactor lands — both files need to be edited in lockstep, and there is no compile-time link between them.

Low-cost polish: lift the fallback into src/lib/site-config.ts (alongside siteName, siteAuthor, the new CONTACT_EMAIL, etc.) and import it. That avoids the assertRequiredEnv indirection that the file-header comments deliberately avoid — a plain string export does not transitively touch process.env in the bundle graph, so the Satori/edge concern those comments cite stays addressed.

Non-blocking — the duplication is two lines, the convergence cost is small, and you may prefer the file-local readability. Flagging because the rest of this PR is consolidating site-identity constants and this is the same theme.

julianken · 2026-05-17T15:36:23Z

@Mergifyio queue

mergify · 2026-05-17T15:36:39Z

* chore(docs): drop seo-strategy folder; align README to renamed post slug Removes docs/seo-strategy/ — research artifacts from the SEO + AI- discovery analysis funnel, no longer load-bearing now that the gate-1/2/3 work has shipped (#393 #394 #395 #396 #397 #400 #402 #404 #406 #408). History preserved in git. README: align "Recent essays" entry with the renamed post slug (where-agentic-patterns-actually-live → agentic-patterns-in-your-coding-workflow). The rename satisfies Bing Site Scan's 70-char title cap. No redirect deployed — article is two days old, no significant external link equity to preserve. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * fix: defer README slug update; gitignore docs/seo-strategy Address julianken-bot review of PR #409: BLOCKER (README:75) — New slug URL serves an SSR 404 fallback because the Payload post slug hasn't been renamed yet (intentionally deferred until the in-flight Bing Site Scan completes). Reverting the README link change here; it will land in a follow-up PR after the actual Payload slug rename, so the link is never broken in main. Plus: add /docs/seo-strategy/ to .gitignore so future analysis-funnel artifacts (phase-*, context-packets, STATUS.md, issues/) stay on disk without polluting the index. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

julianken added status:in-review PR open, waiting for review area:seo SEO + AI-discovery strategy work labels May 17, 2026

julianken mentioned this pull request May 17, 2026

fix(seo): broken sameAs URL + 5 other quick-wins #387

Closed

9 tasks

julianken-bot approved these changes May 17, 2026

View reviewed changes

mergify Bot added the queued label May 17, 2026

mergify Bot merged commit 5d73331 into main May 17, 2026
13 checks passed

mergify Bot deleted the fix/seo-quick-wins branch May 17, 2026 15:37

mergify Bot removed the queued label May 17, 2026

This was referenced May 17, 2026

feat(seo): migrate to Tier C identity (real name + bylines + ProfilePage schema) #388

Closed

feat(seo): Tier C identity migration + ProfilePage schema #400

Merged

julianken-bot mentioned this pull request May 17, 2026

infra(seo): llms.txt + RSS metadata upgrades + site description #394

Merged

5 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(seo): immediate EEAT and crawl-surface fixes#393

fix(seo): immediate EEAT and crawl-surface fixes#393
mergify[bot] merged 1 commit into
mainfrom
fix/seo-quick-wins

julianken commented May 17, 2026

Uh oh!

julianken-bot left a comment

Uh oh!

julianken-bot May 17, 2026

Uh oh!

julianken commented May 17, 2026

Uh oh!

mergify Bot commented May 17, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

julianken commented May 17, 2026

Summary

Changes

Test plan

Coordination

Uh oh!

julianken-bot left a comment

Choose a reason for hiding this comment

Verdict: APPROVE

Verification ledger

Findings

Bottom line

Same-tier note

Uh oh!

julianken-bot May 17, 2026

Choose a reason for hiding this comment

Uh oh!

julianken commented May 17, 2026

Uh oh!

mergify Bot commented May 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Queue Status

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mergify Bot commented May 17, 2026 •

edited

Loading